-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add build specs for Empty and Basic Activity Android Applications and other upgrades #120
Add build specs for Empty and Basic Activity Android Applications and other upgrades #120
Conversation
tresat
commented
Sep 17, 2024
•
edited
Loading
edited
- Adds 2 new project init specs for Empty Activity and Basic Activity Android Applications.
- Upgrades project to use Java 17 (align with Android requirements).
- Upgrades Kotlin version to 2.0.21 (align with Kotlin DSL in Gradle).
- Update to use new Gradle nightly including merged Build Init Specs.
- Various integration testing fixes and additions.
- Updates KMP plugin to hide Polymorphic container, which is no longer supported.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This overall looks good to me.
There are many pending TODOs so I don't approve right away.
Thanks for the review. This has to wait until the init changes on the Gradle side are finished, or at least stable enough that I can build a nightly and update the wrapper here to use it. I don't want to merge this first. |
4a4287f
to
08cd687
Compare
# Conflicts: # unified-prototype/gradle.properties
# Conflicts: # unified-prototype/gradle/wrapper/gradle-wrapper.properties
# Conflicts: # unified-prototype/unified-plugin/plugin-android/src/main/java/org/gradle/api/experimental/buildinit/AndroidProjectSource.java
…egration testing to specifiy type
…udio, fix testing # Conflicts: # unified-prototype/unified-plugin/plugin-jvm/src/main/java/org/gradle/api/experimental/jvm/buildinit/JVMProjectSource.java # unified-prototype/unified-plugin/plugin-kmp/src/main/java/org/gradle/api/experimental/kmp/buildinit/KMPProjectSource.java # Conflicts: # unified-prototype/unified-plugin/plugin-android/src/main/resources/META-INF/services/org.gradle.buildinit.projectspecs.InitProjectSource # unified-prototype/unified-plugin/plugin-jvm/src/main/resources/META-INF/services/org.gradle.buildinit.projectspecs.InitProjectSource # unified-prototype/unified-plugin/plugin-kmp/src/main/resources/META-INF/services/org.gradle.buildinit.projectspecs.InitProjectSource
Polymorphic containers are not supported yet in DCL, so hide this one and expose the (limited) functionality of the known target platform types statically in a non-container interface.
# Conflicts: # unified-prototype/unified-plugin/plugin-common/src/main/java/org/gradle/api/experimental/buildinit/StaticProjectGenerator.java # unified-prototype/unified-plugin/plugin-common/src/main/java/org/gradle/api/experimental/buildinit/StaticProjectSpec.java
08cd687
to
e001341
Compare